Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CS2103-F11-1] ManageEZPZ #62

Open
wants to merge 791 commits into
base: master
Choose a base branch
from

Conversation

Denniszedead
Copy link

@Denniszedead Denniszedead commented Feb 16, 2022

ManageEZPZ helps to keep track of the task assigned and to aid in overseeing employees.

@Denniszedead Denniszedead changed the title Update README.md AY2122S2-CS2103-F11-t-ManageEZPZ Feb 16, 2022
@Denniszedead Denniszedead changed the title AY2122S2-CS2103-F11-t-ManageEZPZ [AY2122S2-CS2103-F11] ManageEZPZ Feb 16, 2022
@Denniszedead Denniszedead changed the title [AY2122S2-CS2103-F11] ManageEZPZ [CS2103-F11-1] ManageEZPZ Feb 16, 2022
@codecov-commenter
Copy link

codecov-commenter commented Mar 6, 2022

Codecov Report

Merging #62 (0c7bd77) into master (4f7660a) will increase coverage by 3.57%.
The diff coverage is 81.01%.

@@             Coverage Diff              @@
##             master      #62      +/-   ##
============================================
+ Coverage     72.15%   75.72%   +3.57%     
- Complexity      399      851     +452     
============================================
  Files            70      107      +37     
  Lines          1232     2604    +1372     
  Branches        125      311     +186     
============================================
+ Hits            889     1972    +1083     
- Misses          311      520     +209     
- Partials         32      112      +80     
Impacted Files Coverage Δ
src/main/java/manageezpz/AppParameters.java 75.00% <ø> (ø)
src/main/java/manageezpz/Main.java 0.00% <ø> (ø)
src/main/java/manageezpz/MainApp.java 0.00% <0.00%> (ø)
src/main/java/manageezpz/commons/core/Config.java 68.18% <ø> (ø)
...main/java/manageezpz/commons/core/GuiSettings.java 69.23% <0.00%> (ø)
.../main/java/manageezpz/commons/core/LogsCenter.java 78.37% <ø> (ø)
...rc/main/java/manageezpz/commons/core/Messages.java 0.00% <0.00%> (ø)
src/main/java/manageezpz/commons/core/Version.java 82.50% <ø> (ø)
...main/java/manageezpz/commons/core/index/Index.java 100.00% <ø> (ø)
...pz/commons/exceptions/DataConversionException.java 100.00% <ø> (ø)
... and 103 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f7660a...0c7bd77. Read the comment docs.

aaron-ljx and others added 26 commits April 6, 2022 21:07
* 'master' of https://github.com/Denniszedead/tp: (34 commits)
  edited UG slightly
  checkstyle fix
  code edited based on danny's suggestion
  checkstyle fix
  checkstyle fix
  added some comments
  fixed bug where task still updates even though empty values were supplied
  fix todo empty description bug
  Remove print line
  Added more guard clause, for editTask, markTask, unmarkTask, tagPriority.
  Fix message success string and remove print line
  Change 'delete' to 'deletes'
  Replace helpMessage picture with right-click to copy feature, edit `help` instruction to include right-click to copy.
  Modify copy clipboard feature title and decrease font size of some labels
  Refractor Help Window image file name
  Modify wrap text property of num of tasks label in Person List card
  Update GUI for Help Window
  Optimize GUI and modify some properties
  Fix Gradle Errors
  Solve JSON modified crashing ManageEZPZ, set-up defensive programming for JsonAdaptedTask
  ...
* master: (34 commits)
  edited UG slightly
  checkstyle fix
  code edited based on danny's suggestion
  checkstyle fix
  checkstyle fix
  added some comments
  fixed bug where task still updates even though empty values were supplied
  fix todo empty description bug
  Remove print line
  Added more guard clause, for editTask, markTask, unmarkTask, tagPriority.
  Fix message success string and remove print line
  Change 'delete' to 'deletes'
  Replace helpMessage picture with right-click to copy feature, edit `help` instruction to include right-click to copy.
  Modify copy clipboard feature title and decrease font size of some labels
  Refractor Help Window image file name
  Modify wrap text property of num of tasks label in Person List card
  Update GUI for Help Window
  Optimize GUI and modify some properties
  Fix Gradle Errors
  Solve JSON modified crashing ManageEZPZ, set-up defensive programming for JsonAdaptedTask
  ...
* Move messages that are only used by EditTaskCommand from Messages.java
to EditTaskCommand.java

* Modify exception message to better reflect invalid time details for
Event task

* Remove String messages that are not in use
Denniszedead and others added 30 commits April 11, 2022 00:42
* Change font size of Employees and Tasks heading from 14pt to 15pt
* Add welcome text message when user launches the application
* Update Ui.png
add in assertEquals, assertFalse and assertTrue statements for better
test coverage
…EmployeeAndTask

Modify test cases for DeleteTask, DeleteEmployee and EditEmployee
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants